chore: restructure codeowners from individuals to teams by pepol · Pull Request #2870 · wundergraph/cosmo

pepol · 2026-05-14T14:41:08Z

chore: refactor codeowners
chore: ignore codeowners changes in ci

Summary by CodeRabbit

Chores
- Consolidated repository ownership into fewer, team-based owners and simplified path mappings across top-level areas.
- Removed many granular ownership entries while preserving the protobuf-generated-files ignore block.
- Updated CI triggers so edits to the ownership declaration no longer run long workflows; a lightweight CI job will run for PRs that modify it.

Checklist

I have discussed my proposed changes in an issue and have received approval to proceed.
I have followed the coding standards of the project.
Tests or benchmarks have been added or updated.
Documentation has been updated on https://github.com/wundergraph/docs-website.
I have read the Contributors Guide.

Open Source AI Manifesto

This project follows the principles of the Open Source AI Manifesto. Please ensure your contribution aligns with its principles.

claude

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

coderabbitai · 2026-05-14T14:41:29Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

Walkthrough

Repository ownership mappings in .github/CODEOWNERS were consolidated to team owners and .github subpaths remapped; five CI workflows were updated to either ignore or include .github/CODEOWNERS in their path filters.

Changes

Repository Ownership Reorganization

Layer / File(s)	Summary
CODEOWNERS Ownership Consolidation `.github/CODEOWNERS`	Consolidated path ownership to team owners; wildcard now lists `@wundergraph/Cosmo` and `@wundergraph/Router`, `/.github/CODEOWNERS` protection owner set to `@wundergraph/Owners`, `.github` subpaths assigned to `@wundergraph/Platform`, and top-level directories consolidated among `@wundergraph/Cosmo`, `@wundergraph/Router`, and `@wundergraph/Platform`.
Workflow Path Filters for CODEOWNERS `.github/workflows/codecov-post-merge.yaml`, `.github/workflows/codeql.yml`, `.github/workflows/helm-release.yaml`, `.github/workflows/release-preview.yaml`, `.github/workflows/dummy-ci.yaml`	Added `.github/CODEOWNERS` to `paths-ignore` for Codecov post-merge, CodeQL, Helm release, and release-preview workflows; added `.github/CODEOWNERS` to dummy CI `pull_request.paths` so dummy CI runs when that file is modified.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

wundergraph/cosmo#2791: Both PRs modify .github/CODEOWNERS ownership mappings for overlapping directory patterns (admission-server, cdn-server, cli, studio, etc.).
wundergraph/cosmo#2856: Both PRs modify .github/CODEOWNERS path-scoped rules for /cli/, /connect/, and /connect-go/.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: restructuring CODEOWNERS from individual assignments to team-based ownership across multiple directories and workflows.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 2

🧹 Nitpick comments (1)

.github/CODEOWNERS (1)
2-2: Consider explicit CODEOWNERS patterns for configuration and metadata files to avoid unnecessary dual-team reviews.

The dual-team wildcard ownership (@wundergraph/Cosmo @wundergraph/Router``) is intentional per the file comment, but it requires approval from both teams for files outside explicit patterns. These unmapped files are predominantly configuration (.prettierrc.json, `.npmrc`, `.gitignore`), GitHub metadata (`.github/ISSUE_TEMPLATE/*`, `.github/labeler.yml`), IDE run configurations, and other non-source files that typically don't need dual-team oversight.

Adding explicit single-team patterns for these categories would streamline reviews without affecting the core source code directories that are already well-specified.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/CODEOWNERS at line 2, Update the .github/CODEOWNERS entry that
currently assigns "* `@wundergraph/Cosmo` `@wundergraph/Router`" by adding explicit
patterns that assign single-team ownership for configuration and metadata files
(e.g., entries targeting .prettierrc.json, .npmrc, .gitignore,
.github/ISSUE_TEMPLATE/*, .github/labeler.yml, IDE run configs like .vscode/* or
*.code-workspace, and other non-source file patterns) so those files are owned
by one team (either `@wundergraph/Cosmo` or `@wundergraph/Router`) while leaving the
wildcard/dual-team line for source directories intact.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/CODEOWNERS:
- Line 1: Update the top comment in .github/CODEOWNERS to correct the typo:
replace the word "expections" with "exceptions" so the line reads "Give everyone
ownership for everything exceptions listed below"; ensure only the comment text
is changed and punctuation remains consistent.
- Around line 2-28: The CODEOWNERS entries reference non-existent GitHub teams
(`@wundergraph/Cosmo`, `@wundergraph/Router`, `@wundergraph/Platform`,
`@wundergraph/Owners`) causing 404s; either create organization teams with those
exact slugs or update the CODEOWNERS file to use the correct existing team slugs
(replace each occurrence of `@wundergraph/Cosmo`, `@wundergraph/Router`,
`@wundergraph/Platform`, `@wundergraph/Owners`) so the ownership rules resolve
properly and stop failing silently.

---

Nitpick comments:
In @.github/CODEOWNERS:
- Line 2: Update the .github/CODEOWNERS entry that currently assigns "*
`@wundergraph/Cosmo` `@wundergraph/Router`" by adding explicit patterns that assign
single-team ownership for configuration and metadata files (e.g., entries
targeting .prettierrc.json, .npmrc, .gitignore, .github/ISSUE_TEMPLATE/*,
.github/labeler.yml, IDE run configs like .vscode/* or *.code-workspace, and
other non-source file patterns) so those files are owned by one team (either
`@wundergraph/Cosmo` or `@wundergraph/Router`) while leaving the wildcard/dual-team
line for source directories intact.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 41608fef-2450-4a20-9d9f-8968d21569cb

📥 Commits

Reviewing files that changed from the base of the PR and between 2be11bc and bcbf93b.

📒 Files selected for processing (1)

.github/CODEOWNERS

Aenimus

LGTM!

…m-individuals-to-teams

pepol requested review from a team, Aenimus and StarpTech as code owners May 14, 2026 14:41

pepol requested a review from ysmolski May 14, 2026 14:41

claude Bot reviewed May 14, 2026

View reviewed changes

thisisnithin requested changes May 14, 2026

View reviewed changes

Comment thread .github/CODEOWNERS Outdated

coderabbitai Bot reviewed May 14, 2026

View reviewed changes

Comment thread .github/CODEOWNERS Outdated

Comment thread .github/CODEOWNERS

thisisnithin approved these changes May 14, 2026

View reviewed changes

Aenimus reviewed May 14, 2026

View reviewed changes

Comment thread .github/CODEOWNERS

Aenimus reviewed May 14, 2026

View reviewed changes

Comment thread .github/CODEOWNERS Outdated

pepol requested a review from Aenimus May 18, 2026 16:11

comatory reviewed May 19, 2026

View reviewed changes

Comment thread .github/CODEOWNERS

comatory approved these changes May 19, 2026

View reviewed changes

Noroth reviewed May 19, 2026

View reviewed changes

Comment thread .github/CODEOWNERS Outdated

Noroth approved these changes May 19, 2026

View reviewed changes

endigma approved these changes May 19, 2026

View reviewed changes

pepol added 7 commits May 19, 2026 15:39

chore: refactor codeowners

6283aad

chore: ignore codeowners changes in ci

bb27fd6

chore: apply dummy ci to codeowners changes

686d402

chore: playground to cosmo

6433d2a

chore: fix typo

6b18e05

chore: add platform as co-owners of cdn

ebda374

chore: remove comment

60c5d7c

pepol force-pushed the peter/eng-9583-github-migrate-codeowners-from-individuals-to-teams branch 2 times, most recently from ca15663 to 60c5d7c Compare May 19, 2026 14:07

Aenimus approved these changes May 20, 2026

View reviewed changes

pepol enabled auto-merge (squash) May 20, 2026 22:47

Merge branch 'main' into peter/eng-9583-github-migrate-codeowners-fro…

16bf49c

…m-individuals-to-teams

pepol merged commit 68e0963 into main May 20, 2026
7 checks passed

pepol deleted the peter/eng-9583-github-migrate-codeowners-from-individuals-to-teams branch May 20, 2026 22:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: restructure codeowners from individuals to teams#2870

chore: restructure codeowners from individuals to teams#2870
pepol merged 8 commits into
mainfrom
peter/eng-9583-github-migrate-codeowners-from-individuals-to-teams

pepol commented May 14, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

claude Bot left a comment

Uh oh!

coderabbitai Bot commented May 14, 2026 •

edited

Loading

Reviews paused

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Aenimus left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

pepol commented May 14, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Checklist

Open Source AI Manifesto

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Claude Code Review

Uh oh!

coderabbitai Bot commented May 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Aenimus left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

pepol commented May 14, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented May 14, 2026 •

edited

Loading